Skip to content

Conversation

@SebastianKrupinski
Copy link
Contributor

@SebastianKrupinski SebastianKrupinski commented Jan 6, 2026

Summary

  • Added processing overlay to editor to visually notify user and prevent user interaction while saving/deleting/responding to event
  • Also moved invitation response logic from Invitation buttons component to editor mixin's to respect separation of logic
image

@SebastianKrupinski
Copy link
Contributor Author

@nimishavijay

Can I have your input on this, please

@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/mixins/EditorMixin.js 0.00% 6 Missing ⚠️

📢 Thoughts on this report? Let us know!

left: 0;
right: 0;
bottom: 0;
background-color: rgba(255, 255, 255, 0.95);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
background-color: rgba(255, 255, 255, 0.95);
background-color: rgba(var(--backdrop-color), 0.5);

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment on lines 1197 to 1202
// Dark mode support
@media (prefers-color-scheme: dark) {
.edit-full__loading-overlay {
background-color: rgba(0, 0, 0, 0.95);
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// Dark mode support
@media (prefers-color-scheme: dark) {
.edit-full__loading-overlay {
background-color: rgba(0, 0, 0, 0.95);
}
}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Copy link
Contributor

@GVodyanov GVodyanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! We have standard colors for this tho

@SebastianKrupinski
Copy link
Contributor Author

Looks good! We have standard colors for this tho

Thanks! Let wait to see what design thinks of this...

@kra-mo @nimishavijay

Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
@SebastianKrupinski SebastianKrupinski force-pushed the fix/editors-processing-overlay branch from e4223cd to 62585ac Compare January 20, 2026 19:54
Copy link
Member

@kra-mo kra-mo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same comment as for #7832: #7832 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

4 participants